Skip to content

security fixes#89

Merged
h3xxit merged 1 commit into
mainfrom
dev
Jun 18, 2026
Merged

security fixes#89
h3xxit merged 1 commit into
mainfrom
dev

Conversation

@h3xxit

@h3xxit h3xxit commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary by cubic

Hardened redirect and header handling across http, gql, and websocket to prevent credential leaks and header injection. Custom auth headers are now stripped on cross‑origin redirects, and CR/LF in header names is rejected.

  • Bug Fixes

    • Strip caller-declared auth headers on cross-origin redirects via new auth_header_names threaded from _apply_auth (covers ApiKeyAuth custom header names and Authorization for OAuth2).
    • Preserve headers on same-origin redirects; added tests to pin behavior.
    • Reject CR/LF in attacker-controlled header fields (e.g., ApiKeyAuth.var_name) to block HTTP header injection in HTTP/SSE/streamable HTTP paths.
    • Implemented in safe_request_with_redirects for utcp-http, utcp-gql, and utcp-websocket; new param is internal-only and not sent on the wire.
  • Dependencies

    • Bump utcp-http to 1.1.7, utcp-gql to 1.1.4, utcp-websocket to 1.1.4.

Written for commit f6f51e9. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 10 files

Re-trigger cubic

@h3xxit h3xxit merged commit 29c0d1b into main Jun 18, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant